Skip to content

fix(snapshot): detect and recover validator vote snapshot inconsisten…#112

Open
On1x wants to merge 8 commits into
masterfrom
snapshot-fix
Open

fix(snapshot): detect and recover validator vote snapshot inconsisten…#112
On1x wants to merge 8 commits into
masterfrom
snapshot-fix

Conversation

@On1x
Copy link
Copy Markdown
Member

@On1x On1x commented May 21, 2026

…cies

  • Add sanity check during export to warn if validators exist but validator votes are absent
  • Log warning about possible chainbase type-enum mismatch causing incomplete snapshot
  • Implement fallback during import to recover validator votes from legacy witness_vote key if validator_vote is empty
  • Improve snapshot integrity by handling potential silent corruption cases due to type enum shifts

On1x added 8 commits May 21, 2026 06:41
…cies

- Add sanity check during export to warn if validators exist but validator votes are absent
- Log warning about possible chainbase type-enum mismatch causing incomplete snapshot
- Implement fallback during import to recover validator votes from legacy witness_vote key if validator_vote is empty
- Improve snapshot integrity by handling potential silent corruption cases due to type enum shifts
…options

- Deleted all mentions of `LOW_MEMORY_NODE` from build scripts, environment variables, and documentation
- Removed low-memory node build instructions and flags from Linux, macOS, and Windows build guides
- Updated CMake options and environment variables to exclude low-memory settings
- Simplified Docker image CMake flags by removing `LOW_MEMORY_NODE`
- Cleared low-memory related config references in node setup and getting started guides
- Cleaned up example config files by removing deprecated plugins and options related to low-memory builds
- Delete config_debug_mongo.ini to clean up obsolete debug mongo configuration
- Remove config_mongo.ini to eliminate outdated mongo production configuration
- Simplify project configuration by removing unused or legacy mongo ini files
- Changed info-level logs (ilog) to debug-level logs (dlog) when connecting to peers and sending DLT hello messages
- Updated rate-limit notification from ilog to dlog for peer exchange requests
- Ensured logging reflects appropriate verbosity level for peer communication events
- Handle CORS preflight by responding to OPTIONS method with proper headers
- Append Access-Control-Allow-Origin header to all HTTP responses
- Add Access-Control-Allow-Methods, Allow-Headers, and Max-Age headers for OPTIONS responses
- Ensure CORS headers are included on error and success responses
- Prevent CORS issues for cross-origin API calls through the webserver plugin
- Add check to skip logging if disconnect is already in progress for a peer
- Avoid re-entrance in send_message calls during handle_disconnect coroutine
- Prevent excessive log entries when send queue is at max depth and peer disconnects
…ad fiber

- Close socket first to unblock pending I/O and avoid multi-second hangs
- Erase connection after closing to prevent dangling shared_ptr references
- Cancel read fiber only after socket is closed to ensure immediate exit
- Retain reentrancy guard to keep peer state valid during disconnect handling
- Adjust order of operations to fix deadlock when multiple peers disconnect simultaneously
- Introduced Ƶ as the short symbol for VIZ chosen by the community
- Explained common practice of showing balances with 2 decimal places
- Noted that even staked funds (SHARES) are displayed as Ƶ with staking notes
- Clarified symbol usage in wallets, explorers, and applications

docs(webserver): document native CORS support in webserver plugin

- Detailed handling of browser cross-origin requests without reverse proxy
- Specified preflight (OPTIONS) response headers and values
- Confirmed all other responses include Access-Control-Allow-Origin: *
- Mentioned compatibility with production setups using nginx proxy
- Highlighted use cases for browser-based wallets and dApps calling JSON-RPC endpoints directly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant